Adwaita: osd spinbutton fixes

Adwaita: stop consider buttons on osd 'linked'

we used to consider every button inside osd containers linked,
this is not true anymore, now those buttons behave normally.
This will clearly cause breakage in applications.

gtkprintoperation: job names must not exceed 255 chars

According to http://datatracker.ietf.org/doc/rfc2911/, The 'name'
attribute syntax is essentially the same as 'text', including the
REQUIRED support of UTF-8 except that the sequence of characters
is limited so that its encoded form MUST NOT exceed 255 (MAX) octets.

CUPS will not print jobs with names exceeding 255 characters.

https://bugzilla.gnome.org/show_bug.cgi?id=755988

im context simple: Add documentation

Document that GtkIMContextSimple is loading additional compose
tables. Also document Ctrl-Shift-u.

GtkIMContextSimple: Load locale compose tables dynamically.

Load /usr/share/X11/locale/$LOCALE/Compose and $HOME/.XCompose
dynamically.

https://bugzilla.gnome.org/show_bug.cgi?id=721120

Add gtk_compose_table_new_with_file() to create GtkComposeTable

https://bugzilla.gnome.org/show_bug.cgi?id=721120

GtkIMContextSimple: Change guint to guint16 in gtkimcontextsimple.c

GtkComposeTable and GtkComposeTableCompact use guint16 for the data.

https://bugzilla.gnome.org/show_bug.cgi?id=721120

GtkIMContextSimple: export _gtk_check_compact_table() and GtkComposeTable

Some functions need to be exported if X11 compose files are loaded.

https://bugzilla.gnome.org/show_bug.cgi?id=721120

win32: use the same parameter name as the implementation

Adwaita: simplify spibutton styling a bit

iconhelper: Track icon-name fallback usage in the GIcon

Passing GTK_ICON_LOOKUP_GENERIC_FALLBACK to the icon lookup doesn't work
for GIcons, so we have to make sure we use the right GThemedIcon.

Fixes image-icon-name-use-fallback reftest.

reftests: Rename reftest

After figuring out what the actual problem is, name the reftest
properly.
The actual problem is that the use-fallback property is ignored when
using an icon-name on GtkImage.

reftests: Add test for GtkImage use-fallback property

Fallback seems to be working in the GtkIconTheme test suite, but fails
in GtkImage itself.

This is a test for a bug in the Bluetooth settings. An icon named
"phone-apple-iphone" should fallback to "phone" if the
gnome-icon-theme-extras package isn't installed.

Small cleanup

Use g_clear_object in gtkdnd.c.

color chooser: Render icons sharp at scale 2

We need to load the icons at the right scale, and render them
to a surface with the right scale too.

https://bugzilla.gnome.org/show_bug.cgi?id=756195

gtkfilesystem: protect against null filesystem type

The filesystem:type attribute could be NULL, then g_strv_contains
will crash if that happens.
Just don't call it if the attribute is not set.

wayland: Allow falling back to compatibility EGL contexts

If the shared context is in legacy mode, or if the creation of a core
profile context failed, we fall back to an EGL context in compatibility
mode.

Since we're relying on a fairly new EGL implementation for Wayland, we
don't fall back to the older EGL API, and instead we always require the
EGL_KHR_create_context extension.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

docs: Improve description of gdk_gl_context_is_legacy()

Explain why this function is available, and why you may need it.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

Allow testglarea to work with legacy GL contexts

Use the 1.30 GLSL shading language for the fragment and vertex shaders.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

Control legacy GL context via environment variable

For testing purposes, we may want to force the creation of legacy GL
contexts via an environment variable.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

x11: Create legacy GLX contexts

If GLX has support for the GLX_ARB_create_context_profile extension,
then we use the GLX_CONTEXT_COMPATIBILITY_PROFILE_BIT_ARB; if it does
not, we fall back to the old glXCreateNewContext() API.

We use the shared GdkGLContext to decide whether the GLX context should
use the legacy bit or not.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

gl: Use older GLSL shaders with legacy contexts

If we're using modern GLSL, then we should stop using deprecated
modifiers, like 'varying' and 'attribute', as well as deprecated global
variables, like 'gl_FragColor'.

On the other hand, with legacy contexts we should be using older GLSL
shaders, to maximize compatibility.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

gl: Store the legacy bit in the GL program data

We need to know if we're using a legacy GL context in various places.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

gdk: Allow querying if a GL context is in legacy mode

We want to have the ability to fall back to legacy GL contexts when
creating them. In order to do so, we need to store the legacy bit on the
GdkGLContext, as well as being able to query it.

Setting the legacy bit from outside GDK is not possible; we cannot
create GL contexts in 3.2 core profile *and* compatibility modes at the
same time, and if we allowed users to select the legacy mode themselves,
it would break the creation of the GdkWindow's paint GL context.

What we do allow is falling back to legacy GL context if the platform
does not support 3.2 core profiles — for instance, on older GPUs or
inside virtualized environments.

We are also going to use the legacy bit internally, to choose which GL
API we can use when drawing GL content.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

textiter: fix bug in _gtk_text_btree_get_iter_at_last_toggle()

If the last tag toggle is the end iter, the function returned the wrong
tag toggle.

This resulted in some bugs where the view wasn't relayout/redrawn
correctly.

The function also always returned TRUE, probably because the return
value is used nowhere. But for consistency with
_gtk_text_btree_get_iter_at_first_toggle(), it's better to keep the
return value, and also because otherwise the function would be wrong (it
doesn't always return a tag toggle, if there is none).

https://bugzilla.gnome.org/show_bug.cgi?id=755413

GtkPopover: Don't apply shape on wayland

Popovers are subsurfaces on wayland so we don't need that, and the shape
messed up hidpi popovers there.

wayland: Don't ignore wl_keyboard.leave if surface is gone

keyboard_handle_leave() might be called with a NULL surface resource
(for example if the surface was destroyed after the event was sent). If
so, we should still deal with the keyboard focus lost event, otherwise
we will both leak (the keyboard_focus GdkWindow reference) and miss
stopping the key repeat timer.

https://bugzilla.gnome.org/show_bug.cgi?id=755927

Fix accessibility not getting initialized in gtk_init() if a default display is already set

_gtk_accessibility_init() only gets called if the default
display changes, but in case gdk_init() is called before gtk_init()
the default display is already set and no property notification occurs.

This can happen quite easily in pygobject where
"from gi.repository import Gdk, Gtk"
will call gdk_init() followed by gtk_init() in the Python overrides.

This fixes it by checking for a default display in all cases.

GdkAppLaunchContext: Set DISPLAY variable for X11

The environment variable DISPLAY makes sense only for X11, so set its
value in the X11 backend.

https://bugzilla.gnome.org/show_bug.cgi?id=754983

assistant: Deal with spontaneous page disappearance

Removing pages from the assistant with gtk_widget_destroy() used
to work. It broke with the recent interposition of a box between
each page and the notebook. Fix this by cleaning up when the box
child is removed.

https://bugzilla.gnome.org/show_bug.cgi?id=756042

Updated German translation

flowbox: correct the behavior with "can-focus"==FALSE

In fact there were two issues:

1. GtkFlowBoxChild with "can-focus"==FALSE should pass the focus
to its child immediately.
2. GtkFlowBox with "can-focus"==FALSE should cease its custom keynav
implementation and fall back to the default GtkContainer behavior
which is more natural.

Thanks to these changes the flow box can act as a better replacement
for GtkGrid and similar containers.

https://bugzilla.gnome.org/show_bug.cgi?id=753371

Bump version to 3.19.0

Otherwise, internal use of new API causes deprecation warnings.

inspector: Add a reset button for settings

This is exercising the new gtk_settings_reset_property api.

https://bugzilla.gnome.org/show_bug.cgi?id=755008

Add gtk_settings_reset_property

This api undoes an application-specific override and makes the
setting follow the system-wide values again.

https://bugzilla.gnome.org/show_bug.cgi?id=755008

style context: Avoid a compiler warning

Bug 753992 - im-quartz discard_preedit segmentation fault

Replace checking if the NSView is really a GdkWindow, which will crash
in the likely event it's not a GObject, with ensuring that the parent
GdkWindow is really a GdkWindowQuartz.

gdk: Add gdk_rectangle_equal()

There's enough users inside GTK to warrant this convenience function.

texttag: add gtk_text_tag_changed()

The function is useful for a GtkTextTag subclass that adds new
properties.

https://bugzilla.gnome.org/show_bug.cgi?id=755416

gdk: Add version macros for 3.20

style context: Formatting fixes

Add a test for gtk_style_context_list_classes

This test failed before the previous commit.

Fix gtk_style_context_list_classes

The function was always returning NULL.

switch: Unclip the slider shadow

This commit arranges things so the shadow of the slider can
extend outside the widget, even if the switch itself has no
shadow.

https://bugzilla.gnome.org/show_bug.cgi?id=755791

fixup

3.18.1

Fix make check

file chooser: Clear busy cursor before showing error

It looks odd to still have a busy cursor while the error
dialog is presented.

file chooser: Improve search focus handling

Move focus to list when search results appear to make it
possible to select the first search result by just hitting
Enter. To keep this from interfering with keynav, we need
to make sure that we still handle Escape to search. And when
search comes up empty, we need to move the focus back to the
entry.

https://bugzilla.gnome.org/show_bug.cgi?id=755926

file chooser: Don't select in the search entry

The stack calls gtk_widget_grab_focus on the last focus widget,
which selects the text in the entry, so we need to make sure to
move the focus there first to keep that from happening.

https://bugzilla.gnome.org/show_bug.cgi?id=755931

app: Warn when trying to add windows on an inert instance

Application windows can only be added after the application has been
started.

https://bugzilla.gnome.org/show_bug.cgi?id=755954

window: ignore resize increments for maximized/fullscreen

Once a window is maximized/fullscreen, resize increments should be
ignored otherwise the window may appear smaller than the screen size.

That also applies to configure requests as well.

https://bugzilla.gnome.org/show_bug.cgi?id=751368

Updated Irish translation

header bar: Show a non-pixellated app icon

Load a hi-dpi icon when we are running with scale=2.

https://bugzilla.gnome.org/show_bug.cgi?id=751787

gdkgears: Use a headerbar

This is a testcase for
https://bugzilla.gnome.org/show_bug.cgi?id=755675

GtkPopover: Make transitions faster

https://bugzilla.gnome.org/show_bug.cgi?id=755528

header bar: Don't leave a dangling pointer

This might be the cause for some crashes, like
https://bugzilla.redhat.com/show_bug.cgi?id=1264558

Use direct GValue access in our marshallers

These are used for every signal emission, so we should
make them as fast as we can.

rounded rect: cache arcs

We are only ever drawing axis-aligned quarter arcs here. By caching
these paths, sincos drops from the profiles for drawing-heavy
workloads.

container: Avoid duplicate calls to should_propagate

We were calling gtk_container_should_propagate_draw
twice for each child. We can avoid this by splitting
out an gtk_container_propagate_draw_internal function.

Drop some redundant cairo_save calls

Almost all callers of _gtk_widget_draw already did their own
cairo_save/restore, so drop the save/restore calls inside
_gtk_widget_draw and instead fix the last caller, gtk_widget_draw,
to do the same.

Trivial: Move a cairo_save call

Move the cairo_save call to where it is needed.

GtkPopover: Remove apply_shape field

This was introduced in 2d9c09d6 alongside a private setter, but that
setter has been removed and the field is always TRUE.

GtkPopover: Don't resize during the transition

Call gtk_popover_update_position instead which will pick up the new
transition_diff value and pass it on to
_gtk_window_set_popover_position, which in turn will move the window
correctly.

https://bugzilla.gnome.org/show_bug.cgi?id=755435

GtkWindow: Don't needlessly resize popovers

Check whether the given popover even changed size in
_gtk_window_set_popover_position. If not, just move its GdkWindow
without calling gtk_widget_queue_resize. Using popover_get_rect here is
still relatively costly, but popover_size_allocate would be doing that
anyway.

https://bugzilla.gnome.org/show_bug.cgi?id=755435

GtkPopover: Remove _get_pointed_to_coords

It's only used once and removing it only adds a single line there.

https://bugzilla.gnome.org/show_bug.cgi?id=755435

Updated Italian translation

Updated Italian translation

Return a value from ::command-line handler

The signal expects a return value, so give it one.

https://bugzilla.gnome.org/show_bug.cgi?id=755692

css style: Accumulate changes in place

This avoids allocating a temporary bitmask, and lets us
avoid some value comparisons altogether.

Add a few more inlined getters

Remove overhead in gdk_window_get_user_data

This function is called very frequently during in GTK+'s drawing
paths, and thus should not carry unexpected costs.

Updated Vietnamese translation

Signed-off-by: Trần Ngọc Quân <vnwildman@gmail.com>

Updated Italian translation

Updated Italian translation

Updated Russian translation

Updated Serbian Latin translation

Updated Serbian translation

bitmask: Use gsize, not size_t

We want to use the same type everywhere to avoid surprises.

bitmask: Don't hardcode 64bit size

This looks like an oversight from "quickly testing a potential fix" and
then forgetting to make a production-ready when it works.

https://bugzilla.gnome.org/show_bug.cgi?id=755691

Updated Hebrew translation

Avoid copying lists during draw

We can use gdk_window_peek_children here, instead of copying
the list. Note that we preserve the bottom-to-top ordering by
iterating the list from the end.
gdk_window_get_children_with_user_data was doing a list
reversal while filtering the list.

Refactor gtk_widget_add_events_internal_list

Avoid copying the child window lists. It is not necessary.

Refactor gtk_widget_set_device_enabled_internal

Instead of constructing a single list of all windows associated
to a widget, just iterate the children lists recursively, avoiding
allocations.

Fix Uyghur translation of default:mm

Fix Tajik translation of default:mm

Fix Khmer translation of default:mm

Fix Khmer translation of default:LTR

Fix Kikongo translation of default:mm

Fix Kikongo translation of default:LTR

Fix Bosnian translation of default:mm

gtkbookmarksmanager: don't allow non valid utf8 in bookmarks

In case some client send to us a non valid utf8 string, don't screw up
the bookmarks file and just return.

https://bugzilla.gnome.org/show_bug.cgi?id=755215

gtkplacessidebar: avoid to use a freed string

The string we were using is the representation of the internal text
in the popover entry. However that can be freed before setting the
bookmark label, if i.e. the row is destroyed and therefore the popover
as well.
To avoid that, duplicate the label in a local variable.

One of the consequences is that for those people using development version
we migth screwed its bookmarks file, since the bookmark manager wrote
garbage from the already freed label.

https://bugzilla.gnome.org/show_bug.cgi?id=755215

build/Makefile-newvs.am: Update Comments for Usage

Update the notes that this is also used for Visual Studio 2015 support,
and correct the MSVC_VER_LONG for MSVC 2015, which is 14, not 2015.

Also add a note that this can be used for other projects that have
Visual Studio build support.

Build: Make Makefile.msvcproj A Bit More Generic

Handle also the situation where $(srcdir) == $(top_srcdir), so that this
can also be used in cases like librsvg and gobject-introspection.

textview: Look up the drop location iter on the right coordinates on DND

We are passing widget coordinates to gtk_text_view_window_to_buffer_coords()
which expects coordinates to be relative to the text window in this case.
This may result in drop coordinates being displaced if the side windows to
the top/left sides are visible and taking space, so the DnD indicator will
point to the wrong position.

This can be seen on gnome-builder and gedit when displaying line numbers.

Updated German translation

Don't mix in user options of printer instances

GTK+ currently does not support printer instances in the printing
dialog. This commit avoids user options set for specific instances of
a printer being accidently applied to the printer with the same name but
no specific instance defined.

https://bugzilla.gnome.org/show_bug.cgi?id=753628
https://bugzilla.gnome.org/show_bug.cgi?id=582747

build: Dist toarray.pl

Some distributions (MSYS2, Debian) call autoreconf on a tarball because
they change the autotools artifacts.

In order to rebuild the Broadway generated files, we need to ship the
scripts that we use when disting a release.

HC: use round buttons on sidebars

Copy Adwaita in how sidebar icon buttons look

https://bugzilla.gnome.org/show_bug.cgi?id=755268